Skip to content

extracted Rule from settings.h to rule.h / added Regex::engine() / removed unnecessary Rule::engine#8580

Merged
firewave merged 3 commits into
cppcheck-opensource:mainfrom
firewave:iwyu-xxxx
Jun 2, 2026
Merged

extracted Rule from settings.h to rule.h / added Regex::engine() / removed unnecessary Rule::engine#8580
firewave merged 3 commits into
cppcheck-opensource:mainfrom
firewave:iwyu-xxxx

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

No description provided.

@firewave firewave force-pushed the iwyu-xxxx branch 4 times, most recently from 63e488c to 2fa74c5 Compare May 31, 2026 08:01
@firewave firewave marked this pull request as ready for review May 31, 2026 08:01
Comment thread lib/settings.h Fixed
@firewave firewave marked this pull request as draft May 31, 2026 08:31
Comment thread lib/settings.h
~Settings();

Settings(const Settings&);
Settings& operator=(const Settings&);
Comment thread lib/settings.h
Settings& operator=(const Settings&);

Settings(Settings&&) noexcept;
Settings& operator=(Settings&&) noexcept;
@firewave firewave changed the title extracted Rule from settings.h to rule.h extracted Rule from settings.h to rule.h / added Regex::engine() / removed unnecessary Rule::engine May 31, 2026
@firewave firewave changed the title extracted Rule from settings.h to rule.h / added Regex::engine() / removed unnecessary Rule::engine extracted Rule from settings.h to rule.h / added Regex::engine() / removed unnecessary Rule::engine` May 31, 2026
@firewave firewave changed the title extracted Rule from settings.h to rule.h / added Regex::engine() / removed unnecessary Rule::engine` extracted Rule from settings.h to rule.h / added Regex::engine() / removed unnecessary Rule::engine May 31, 2026
@firewave firewave marked this pull request as ready for review May 31, 2026 15:24
@firewave firewave merged commit 01e2254 into cppcheck-opensource:main Jun 2, 2026
70 checks passed
@firewave firewave deleted the iwyu-xxxx branch June 2, 2026 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants